Skip to content

Export isFrameProcessor helper#611

Open
lukasIO wants to merge 4 commits intomainfrom
lukas/non-breaking-fp
Open

Export isFrameProcessor helper#611
lukasIO wants to merge 4 commits intomainfrom
lukas/non-breaking-fp

Conversation

@lukasIO
Copy link
Contributor

@lukasIO lukasIO commented Feb 13, 2026

also removes the breaking change of #608 .

This version isn't as clean but it's sufficiently strict in typing as long as the input type is defined correctly.

@lukasIO lukasIO requested a review from 1egoman February 13, 2026 13:15
@changeset-bot
Copy link

changeset-bot bot commented Feb 13, 2026

⚠️ No Changeset found

Latest commit: 4ad9892

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes changesets to release 6 packages
Name Type
@livekit/rtc-node Patch
@livekit/rtc-node-darwin-arm64 Patch
@livekit/rtc-node-darwin-x64 Patch
@livekit/rtc-node-linux-arm64-gnu Patch
@livekit/rtc-node-linux-x64-gnu Patch
@livekit/rtc-node-win32-x64-msvc Patch

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 potential issue.

View 3 additional findings in Devin Review.

Open in Devin Review

Copy link
Contributor

@1egoman 1egoman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this makes sense, so instead of looking at the type symbol to infer that on the fly you are using an explicit generic.

Also, that devin warning I think is probably worth looking at as well!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants